Skip to content
This repository has been archived by the owner on Sep 11, 2024. It is now read-only.

Add a button to reset personal encryption state during login #5819

Merged
merged 7 commits into from
Apr 13, 2021

Conversation

turt2live
Copy link
Member

@turt2live turt2live commented Mar 30, 2021

No public issue.


Note: This deliberately gets code review first as changes might affect how the design review goes. This needs design review to land.

Also fair warning I copy/pasted most of this in an effort to reuse existing flows.


image
image

... leads to:
image

@turt2live turt2live requested a review from a team March 30, 2021 21:42
Copy link
Collaborator

@jryans jryans left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this overall looks good code-wise, and should help more people get out of tricky spots on their own. 😄

const cli = MatrixClientPeg.get();
await cli.bootstrapCrossSigning({
authUploadDeviceSigningKeys: async (makeRequest) => {
// XXX: Making this an import breaks the app.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, it works fine in other components... 🤔

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This dialog is loaded async for some reason, which makes it special

@jryans jryans requested a review from a team April 1, 2021 11:53
@turt2live turt2live requested a review from jryans April 9, 2021 00:09
@turt2live turt2live requested a review from jryans April 12, 2021 20:57
@turt2live turt2live merged commit 7bd5c19 into develop Apr 13, 2021
@turt2live turt2live deleted the travis/reset-passphrase branch April 13, 2021 18:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants